Skip to content

fix: restore hourly product workflow YAML parsing - #71

Closed
seonghobae wants to merge 1 commit into
mainfrom
fix/hourly-workflow-yaml-printf-escapes
Closed

fix: restore hourly product workflow YAML parsing#71
seonghobae wants to merge 1 commit into
mainfrom
fix/hourly-workflow-yaml-printf-escapes

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Incident

hourly-product-development.yml has been generating zero-second push failures with no jobs. The workflow only declares schedule and workflow_dispatch, so these runs are GitHub workflow-startup failures rather than product-test failures.

Root cause is two shell printf format strings whose intended \n escapes were stored as physical newlines. Their closing quote/redirection lines begin at column 1, terminating the YAML block scalar and making the workflow unparseable. The first malformed pair is at current lines 119–120; the second is at 235–236.

Test-first state

This draft currently contains only a focused regression test requiring both printf commands to remain single, indented YAML lines with literal \n format escapes. It is intentionally RED against the current workflow. The minimal workflow repair will follow after the expected failure is recorded.

Scope

  • .github/workflows/hourly-product-development.yml
  • tests/test_hourly_opencode_nvidia_contract.py

No model, permission, credential, endpoint, publication, or autonomous-maintenance behavior is being changed.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 17098658-2dea-4a47-b237-141a8fe3b91c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

Copy link
Copy Markdown
Contributor Author

Closing as superseded by PR #66. Exact current head c038a9509d1a8eae8561cc9081e67e12bd373d42 already preserves both affected printf format strings as single indented YAML lines with literal \n escapes, while also removing the repository-local write/publish path that this main-based PR would otherwise restore to executable form. PR #66 has exact-head CI, SAST, and Security Scan success; it remains draft pending exact-current-head review, independent approval, and branch protection. The RED evidence on this PR remains useful regression history, but carrying a second workflow branch would duplicate and weaken the safer trust-boundary repair.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant